auto merge of #470 : alexcrichton/cargo/cargo-new-git, r=brson

This adds a command-line --no-git option to disable this behavior, as well as
adding a global config section for `git = false`. While I was at it I write some
documentation for the configuration format that cargo uses.

Turn --git on by default for `cargo-new`.

This adds a command-line --no-git option to disable this behavior, as well as
adding a global config section for `git = false`. While I was at it I write some
documentation for the configuration format that cargo uses.

auto merge of #507 : alexcrichton/cargo/profiles, r=brson

For documentation, see the included documentation in the commit.

Re-enable passing -g to rustc

Now that there is a way to disable debuginfo for a build, we can go back to
passing it by default. Any bugs in debuginfo will get weeded out by specifying
`debug = false` in the profile.

Implement manifest profiles

For documentation, see the included documentation in the commit.

auto merge of #506 : alexcrichton/cargo/sooner-lockfile, r=wycats

It's quite annoying if you update a dependency, but it takes you awhile to get
the dependency building. Previously the dependency graph would have to be
updated each time because the lockfile was only written *after* a successful
build.

Other tools like `cargo generate-lockfile` will already generate a lockfile at
any time, so just make it easier by moving it up in the compilation process.

Generate a lockfile sooner in `cargo build`

It's quite annoying if you update a dependency, but it takes you awhile to get
the dependency building. Previously the dependency graph would have to be
updated each time because the lockfile was only written *after* a successful
build.

Other tools like `cargo generate-lockfile` will already generate a lockfile at
any time, so just make it easier by moving it up in the compilation process.

auto merge of #501 : alexcrichton/cargo/issue-480, r=brson

If a repository is corrupt, blow it away entirely and start from scratch instead
of returning an error.

Closes #480

auto merge of #500 : alexcrichton/cargo/fix-flaky-test, r=brson

This test has been flaky on the bots for quite some time now, and the cause has
now been discovered. The root cause of the failure is that the execve for the
`cargo --list` command was failing with ETXTBUSY. In querying the manpage, this
means:

    Executable was open for writing by one or more processes.

This error can be explained by the following trace:

1. Thread A, running the `cargo --list` test, opens the destination executable
   for writing because it's copying the current executable into a different
   location.
2. Thread B, some other test, forks the process. The file descriptor of the
   destination executable of thread A is now duplicated in this process.
3. Thread A closes all files and such, and then goes to fork/exec
   `cargo --list`.
4. Thread B has not had time to close all its descriptors. so it still has the
   executable open for writing, causing the `execve` of thread A to fail.

This commit just removes these tested portions of the test, only testing that
cargo probes PATH.

auto merge of #499 : alexcrichton/cargo/issue-486, r=brson

They're not really related to benchmarks!

Closes #486

auto merge of #495 : alexcrichton/cargo/issue-492, r=brson

Closes #492

auto merge of #475 : alexcrichton/cargo/issue-458, r=brson

Relative paths are now considered relative to the directory containing the
`.cargo/config` file specifying the relative path.

I also merge ConfigValueValue into ConfigValue by moving the Path directly next
to the String that defined it so we can track which strings came from which
paths.

Closes #458

auto merge of #491 : alexcrichton/cargo/issue-481, r=brson

The return value of `err.output()` will be `None` if the rustdoc executable
failed to spawn, in which case we can give a more graceful error message.

Closes #481

auto merge of #471 : alexcrichton/cargo/issue-449, r=brson

This is often useful for picking up things like headers files from `*-sys`
packages when they had to compile locally (or perhaps for pkg-config).

Closes #449

auto merge of #469 : alexcrichton/cargo/issue-442, r=brson

Discovering the prefix/suffix needs to understand that it's not actually
available.

Closes #442

Pass native output directories of dependencies

This is often useful for picking up things like headers files from `*-sys`
packages when they had to compile locally (or perhaps for pkg-config).

Closes #449

Fix relative override paths

Relative paths are now considered relative to the directory containing the
`.cargo/config` file specifying the relative path.

I also merge ConfigValueValue into ConfigValue by moving the Path directly next
to the String that defined it so we can track which strings came from which
paths.

Closes #458

Don't fail!() when rustdoc fails to run

The return value of `err.output()` will be `None` if the rustdoc executable
failed to spawn, in which case we can give a more graceful error message.

Closes #481

auto merge of #468 : alexcrichton/cargo/issue-404, r=brson

Whenever possible, try to print a short path by using path_relative_from.

Closes #404

auto merge of #502 : alexcrichton/cargo/try-to-fix-travis, r=alexcrichton

Fix the 32-bit linux travis bots

Slim down the `cargo --list` test

This test has been flaky on the bots for quite some time now, and the cause has
now been discovered. The root cause of the failure is that the execve for the
`cargo --list` command was failing with ETXTBUSY. In querying the manpage, this
means:

    Executable was open for writing by one or more processes.

This error can be explained by the following trace:

1. Thread A, running the `cargo --list` test, opens the destination executable
   for writing because it's copying the current executable into a different
   location.
2. Thread B, some other test, forks the process. The file descriptor of the
   destination executable of thread A is now duplicated in this process.
3. Thread A closes all files and such, and then goes to fork/exec
   `cargo --list`.
4. Thread B has not had time to close all its descriptors. so it still has the
   executable open for writing, causing the `execve` of thread A to fail.

This commit just removes these tested portions of the test, only testing that
cargo probes PATH.

Don't run doc tests for `cargo bench`

They're not really related to benchmarks!

Closes #486

Update docopt to fix -- option parsing

Closes #492

Support cross compiling to triples without dylibs

Discovering the prefix/suffix needs to understand that it's not actually
available.

Closes #442

Print longer paths to Cargo.toml on failure

Whenever possible, try to print a short path by using path_relative_from.

Closes #404

Blow away corrupt database repos

If a repository is corrupt, blow it away entirely and start from scratch instead
of returning an error.

Don't use bundled fonts

auto merge of #487 : Insanity133/cargo/patch-2, r=alexcrichton

This is a fixed version of PR #478 on the gh-pages branch.

Fix a minor typo

auto merge of #485 : alexcrichton/cargo/fix-for-rustc-master, r=alexcrichton

Extending #479 to use a `'static` shell instead of a `'a` shell.

Update git2

This updates git2-rs to not use pkg-config on windows.

Closes #472

Be compatible with the stdlib for now ('static shell)

Refresh Cargo.lock

This brings in versions of upstream components with build break fixes

Fix build breakage due to generalized type parameter and object bounds

It is worth noting that cargo was actually creating trait boxes
that contain non-static references, so I couldn't just put 'static
everywhere to appease rustc.

Fix deprecated method/function warnings with rustc master

auto merge of #416 : alexcrichton/cargo/dox, r=wycats

Closes #414

auto merge of #465 : alexcrichton/cargo/update-git2, r=wycats

It turned out most of the methods in libgit2 don't actually require a Signature
structure, they're all mostly optional. This commit updates to this version of
libgit2 where the arguments are all optional.

Closes #463

Update the git2 dependency

It turned out most of the methods in libgit2 don't actually require a Signature
structure, they're all mostly optional. This commit updates to this version of
libgit2 where the arguments are all optional.

auto merge of #459 : alexcrichton/cargo/update-git, r=wycats

An explanation can be found in the commit I made to git2-rs [1]. This is the
first step towards dealing with #457.

[1]: https://github.com/alexcrichton/git2-rs/commit/16142ef9

Link to openssl statically for nightlies

An explanation can be found in the commit I made to git2-rs [1]. This is the
first step towards dealing with #457.

[1]: https://github.com/alexcrichton/git2-rs/commit/16142ef9

auto merge of #453 : dotdash/cargo/glob, r=alexcrichton

The glob crate that comes with rust itself has been deprecated.

auto merge of #455 : alexcrichton/cargo/cargo-run, r=alexcrichton

Just some minor test fixes.

Fix cargo-run rebasings and tests

Closes #443

Merge remote-tracking branch 'cmr/master'

Conflicts:
src/cargo/ops/cargo_run.rs
tests/test_cargo_run.rs

Fix building with current rust by using rust-lang/glob

The glob crate that comes with rust itself has been deprecated.

auto merge of #452 : alexcrichton/cargo/fix-nightly-linux, r=alexcrichton

The linux nightly failed and hopefully this will fix it.

Bump the git2 dep

The linux nightly failed and hopefully this will fix it.

auto merge of #446 : alexcrichton/cargo/libgit2-again, r=brson

I applied https://github.com/libgit2/libgit2/pull/2414 to a [local branch](https://github.com/alexcrichton/libgit2/tree/mingw-w64-winhttp) and then fixed the build for mingw-w64. The tests pass fine on mingw and I've used https successfully on windows with libgit2 on cargo now.

Roll back git2 a bit to fix tests

Update git2-rs with mingw support

An upstream PR for libgit2 was applied locally, modified to build, and then
git2-rs was updated to use this small for of libgit2. This adds https support
via winhttp on mingw for windows users.

Revert "Revert "Use libgit2 for driving git instead of the CLI""

This reverts commit 04440bb035082d4163f2211af447b8d67e077ccb.

Conflicts:
src/cargo/sources/git/utils.rs

auto merge of #444 : alexcrichton/cargo/issue-54, r=brson

This generates errors for malformed semver versions during the decoding process
rather than later in the convertion to a package id. This also cuts down on the
large number of derived traits to only what's necessary.

Closes #54

Disable travis on all branches but master

auto merge of #441 : EduardoBautista/cargo/fix-inconsistencies, r=alexcrichton

The guide appeared to be inconsistent with its use of "0.1.0" and "0.0.1" so I decided to change all version numbers to "0.0.1".

auto merge of #423 : alexcrichton/cargo/cargo-package, r=brson

This command will assemble the current package into a tarball ready for
uploading to the cargo registry. Currently no further verification is done
beyond packaging the local repository into a tarball, but in the future this
could execute other operations such as api stability tools.

Implement cargo-package

This command will assemble the current package into a tarball ready for
uploading to the cargo registry. Currently no further verification is done
beyond packaging the local repository into a tarball, but in the future this
could execute other operations such as api stability tools.

auto merge of #435 : alexcrichton/cargo/issue-431, r=brson

This option is used to disable the --test flag to rustc for a test or benchmark
target in order to signal that the binary already knows how to run the testing
infrastructure.

The test/benchmark is still compiled and run as usual, and the exit code is
expected to reflect the result of the test/benchmark.

Closes #431

Help debug a flaky windows test

Add a `harness` manifest option

This option is used to disable the --test flag to rustc for a test or benchmark
target in order to signal that the binary already knows how to run the testing
infrastructure.

The test/benchmark is still compiled and run as usual, and the exit code is
expected to reflect the result of the test/benchmark.

Closes #431

auto merge of #420 : alexcrichton/cargo/issue-393-2, r=brson

This commit removes all distributed executables except for `cargo`. All
builtin subcommands are implemented through library calls, and the fallback
methods are retained to maintain extensability through new subcommands.

Closes #393

Remove all subcommand executables

This commit removes all distributed executables except for `cargo`. All
builtin subcommands are implemented through library calls, and the fallback
methods are retained to maintain extensability through new subcommands.

Closes #393

auto merge of #421 : alexcrichton/cargo/issue-418, r=wycats

auto merge of #419 : alexcrichton/cargo/issue-406, r=brson

This allows tests to be built, but not run.

Closes #406

Fix `cargo run` for renamed bin targets

Add a --no-run option to `cargo test`

This allows tests to be built, but not run.

auto merge of #450 : alexcrichton/cargo/fix-cargo, r=alexcrichton

Fix a flaky test while the compiler is changing

Print out the rustc version before each build

Fix building with current rust

Decode directly into a semver::Version

This generates errors for malformed semver versions during the decoding process
rather than later in the convertion to a package id. This also cuts down on the
large number of derived traits to only what's necessary.

Closes #54

Use "0.0.1" instead of "0.1.0" for version numbers

cargo-run: support target and release the same as cargo-build

auto merge of #434 : alexcrichton/cargo/issue-433, r=huonw

All subprocesses will now be invoked with CARGO_MANIFEST_DIR pointing at the root of the
source directory that they are working on (compiling). This commit also
reorganizes the version environment variables to use the new infrastructure.

Closes #433

Add a new CARGO_MANIFEST_DIR environment variable

All subprocesses will now be invoked with CARGO_MANIFEST_DIR pointing at the
root of the source directory that they are working on (compiling). This commit
also reorganizes the version environment variables to use the new
infrastructure.

Closes #433

auto merge of #430 : aochagavia/cargo/mod, r=alexcrichton

Example:

```
use std::fmt;
use std::fmt::Show;
```

becomes

```
use std::fmt::{mod, Show};
```

Use `mod` where possible

Example:

```
use std::fmt;
use std::fmt::Show;
```

becomes

```
use std::fmt::{mod, Show};
```

auto merge of #428 : garrison/cargo/patch-3, r=alexcrichton

travis-ci now supports "language: rust", which eliminates the need to call rustup.sh in .travis.yml. See http://www.reddit.com/r/rust/comments/2ecocp/travis_ci_supports_rust/ for details.

This is the same change as #425, but on the master branch in guide.md, not guide.html.

Update guide.md to use "language: rust" in .travis.yml

travis-ci now supports "language: rust", which eliminates the need to call rustup.sh in .travis.yml. See http://www.reddit.com/r/rust/comments/2ecocp/travis_ci_supports_rust/ for details.

Beef up some various documentation

auto merge of #415 : alexcrichton/cargo/in-tree-docs, r=wycats

There's a bunch of travis config now to rebuild the website whenever something is merged into master.

Add all docs in-tree

auto merge of #410 : alexcrichton/cargo/fix-doctest-with-dep, r=wycats

The movement of tests to the main `target` directory forgot to update a spot
where rustdoc used a -L flag.

Closes #411

auto merge of #412 : ScriptDevil/cargo/default-help, r=alexcrichton

Most modern command line utilities print usage/help when invoked without any arguments.

cargo (prior to this patch) defaults to saying "no such subcommand" when invoked without arguments.
> $ cargo
> No such subcommand

With this patch, it will instead print the help message
> $ cargo
> Rust's package manager
>
> Usage:
> ...

Print help message if cargo is invoked with no arguments

Fix doc tests for libs with deps

The movement of tests to the main `target` directory forgot to update a spot
where rustdoc used a -L flag.

auto merge of #409 : alexcrichton/cargo/snapshots, r=alexcrichton

Register new snapshots

auto merge of #408 : alexcrichton/cargo/fix-master, r=alexcrichton

Fix some flaky tests on the buildbots

auto merge of #390 : alexcrichton/cargo/revert-libgit2, r=wycats

It has been discovered that the windows builders aren't building a libgit2 with support for SSL, so cloning https repos on windows is broken with libgit2. It doesn't look easy to sort out, so in the meantime this PR reverts the switch to libgit2 and adds @wycats's fixes from the related PR.

Syncs submodules after remote update (closes #370)

Previously, the Git code avoided cloning a local copy of the local
database for a repository if the repo already existed (and instead tried
to fetch). However, fetching does not sync and reinitialize submodules,
which would require more work.

To avoid this problem, and possibly other subtle updating issues in the
future, the code now wipes the local clone whenever it detects that the
HEAD of a particular branch has changed. This avoids subtle
inconsistencies between fresh clones and updates.

Note: This only affects local-to-local clones. It does not initiate any
new network activity. It may require some additional disk usage, but
only when a remote fetch that was already happening discovers new
commits on the checked-out branch.

Revert "Use libgit2 for driving git instead of the CLI"

This reverts commit 3cd82ed7a191c4adb954704e9396368863eccddf.

Conflicts:
src/cargo/sources/git/utils.rs

auto merge of #384 : alexcrichton/cargo/issue-379, r=wycats

Mainly targeted at incremental build times.

Make a timing test a little less flaky

Allow updating transitive deps

Previously `cargo update foo` would only update the package `foo` if it were a
direct dependency of the local package. This meant that to update a transitive
dependency you would have to update the top-level dependency.

This commit adds the ability to update any dependency by name, regardless of
where it is in the dependency graph. This commit is a bit lossy in terms of
behavior. We are guaranteed that the set of immediate dependencies for any one
package have unique names, but not for the entire package graph. This means that
when you invoke `cargo update foo` it could possibly update two packages named
`foo`.

I believe this behavior to be acceptable for now and we can add a more stringent
update syntax later (something like `cargo update --namespace foo bar`). I
believe we'll always want this CLI usage, however.

Preserve $OUT_DIR across rebuilds

The original choice for completely destroying $OUT_DIR was to focus on
repeatable builds to ensure that stale items in $OUT_DIR don't affect later
compilations. This commit moves this responsibility from cargo to the build
command itself.

Build commands are now responsible for cleaning out old artifacts and ensuring
that when invoked the state of $OUT_DIR is as it would be if it started with an
empty $OUT_DIR. This will allow for very long build commands based on `make` to
have a much faster incremental build time as they won't have to rebuild
everything when updated.

This can cause non-repeatable builds with build commands if stale artifacts are
never removed (but still used in the rust compilation step), but this will now
be considered a bug in the build command rather than for cargo itself.

Closes #382

Add an exclude key to the manifest

This key is used to help determine the set of input files for a package. The
normal method (via walking or git ls-files) is filtered via all the patterns
provided in `exclude` of the project section.

The toml key is a string array corresponding to a set of glob patterns according
to the syntax of libglob (provided in the standard distribution). The set of
files normally found will be filtered by each pattern, and if any pattern
matches then the path is excluded.

This will later on be used for `cargo package` when generating a tarball to get
uploaded.

Factor in .gitignore for build cmd freshness

When testing the freshness of a build command, the fingerprint of an entire
package is generated. The current method of fingerprinting a path source is to
just stat() the entire tree and look at mtimes. This works fine when the build
command places *all* output in the build directory.

Some systems, like autotools, place *most* output in the build directory and
some output in the source directory, however (see spidermonkey). In this case
the coarse-grained "consider all files in a directory as input" is too painful
for the build command as the package will forever be rebuilt.

This commit adds support for filtering the list of files in a directory
considered to be part of a package by using `git ls-files`. This git-specific
logic can be extended to other VCSs when cargo grows support for them.

Closes #379

auto merge of #401 : alexcrichton/cargo/issue-348, r=wycats

This commit changes the hash of Profile to only take into account
flags/variables that affect the actual output file itself (as opposed to its
location), and then changes cargo {test, build, doc} to all use the same
directory of output (in order to share deps).

This will cause a `cargo build` to remove all of the tests generated by `cargo
test`, but it speeds up the cycle of `cargo test` followed by a `cargo build` by
not needing to rebuild all dependencies.

Additionally, `cargo bench` now shares the same directory as
`cargo build --release` for the same reasons as above.

Closes #348